Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade ant icons to 5.4 #8007

Merged
merged 5 commits into from
Aug 19, 2024
Merged

Upgrade ant icons to 5.4 #8007

merged 5 commits into from
Aug 19, 2024

Conversation

normanrz
Copy link
Member

@normanrz normanrz commented Aug 16, 2024

Upgrades ant icons to 5.4 matching the version of the main antd package.
Also, ant-icons now use inline svg instead of iconfonts, so we can remove our custom handling of the iconfont.

  • Changelog

@normanrz normanrz self-assigned this Aug 16, 2024
package.json Outdated
@@ -1,17 +1,14 @@
{
"name": "webknossos",
"license": "AGPL-3.0",
"engines":
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in another pr, I also had the problem that the package.json was entirely reformatted. later I had annoying merge conflicts in other prs. could you try to undo these changes?
the proper solution will be to update biome and then let it run on package.json, too. the current version will always skip package.json (see biomejs/biome#1668).
updating requires lots of small fixes, though, as biome got smarter. so, this should be a different pr.

@normanrz normanrz enabled auto-merge (squash) August 19, 2024 12:51
@normanrz normanrz merged commit 5ffd633 into master Aug 19, 2024
2 checks passed
@normanrz normanrz deleted the ant-icons-5.4 branch August 19, 2024 12:59
dieknolle3333 pushed a commit that referenced this pull request Sep 2, 2024
* upgrade ant icons to 5.4

* remove iconfont

* changelog

* reformat package.json

---------

Co-authored-by: Philipp Otto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants